app/components/Package/Dependencies.vue (1)

87-103: Consider extracting the repeated visible-item limit into a single constant.

Using 10 three times makes future tuning error-prone. A shared constant keeps the three sections aligned.

Suggested refactor

+const DEPENDENCY_VISIBLE_LIMIT = 10
+
 const {
   visibleItems: visibleDeps,
   hasMore: hasMoreDeps,
   expand: expandDeps,
-} = useVisibleItems(sortedDependencies, 10)
+} = useVisibleItems(sortedDependencies, DEPENDENCY_VISIBLE_LIMIT)
 
 const {
   visibleItems: visiblePeerDeps,
   hasMore: hasMorePeerDeps,
   expand: expandPeerDeps,
-} = useVisibleItems(sortedPeerDependencies, 10)
+} = useVisibleItems(sortedPeerDependencies, DEPENDENCY_VISIBLE_LIMIT)
 
 const {
   visibleItems: visibleOptionalDeps,
   hasMore: hasMoreOptionalDeps,
   expand: expandOptionalDeps,
-} = useVisibleItems(sortedOptionalDependencies, 10)
+} = useVisibleItems(sortedOptionalDependencies, DEPENDENCY_VISIBLE_LIMIT)

test/unit/app/composables/use-visible-items.spec.ts (2)

6-40: Consider parameterising the initial-state tests with it.each.

The four cases share the same arrange/assert pattern; a table-driven structure would reduce duplication and make future scenarios easier to add.

♻️ Example refactor

 describe('initial state', () => {
-  it('returns all items when list is within limit', () => {
-    const { visibleItems, hasMore, hiddenCount } = useVisibleItems(['a', 'b', 'c'], 5)
-    expect(visibleItems.value).toEqual(['a', 'b', 'c'])
-    expect(hasMore.value).toBe(false)
-    expect(hiddenCount.value).toBe(0)
-  })
+  it.each([
+    { name: 'within limit', items: ['a', 'b', 'c'], limit: 5, visible: ['a', 'b', 'c'], hasMore: false, hidden: 0 },
+    { name: 'exceeds limit', items: [1,2,3,4,5,6,7,8,9,10], limit: 5, visible: [1,2,3,4,5], hasMore: true, hidden: 5 },
+    { name: 'equals limit', items: [1,2,3], limit: 3, visible: [1,2,3], hasMore: false, hidden: 0 },
+    { name: 'empty list', items: [], limit: 5, visible: [], hasMore: false, hidden: 0 },
+  ])('returns correct state when $name', ({ items, limit, visible, hasMore, hidden }) => {
+    const state = useVisibleItems(items, limit)
+    expect(state.visibleItems.value).toEqual(visible)
+    expect(state.hasMore.value).toBe(hasMore)
+    expect(state.hiddenCount.value).toBe(hidden)
+  })
 })

---

103-146: Add explicit boundary tests for non-positive limits.

Please add contract tests for limit = 0 and negative limits (either clamped behaviour or thrown error), so this edge behaviour is pinned down and protected from regressions.

As per coding guidelines, "Write unit tests for core functionality using vitest."